Skip to content

improve: prepare for removal of exitOnStopLeading from public API#3161

Merged
metacosm merged 2 commits intonextfrom
leader-election
Feb 6, 2026
Merged

improve: prepare for removal of exitOnStopLeading from public API#3161
metacosm merged 2 commits intonextfrom
leader-election

Conversation

@metacosm
Copy link
Copy Markdown
Collaborator

@metacosm metacosm commented Feb 6, 2026

Signed-off-by: Chris Laprun [email protected]

@metacosm metacosm self-assigned this Feb 6, 2026
@metacosm metacosm requested a review from csviri February 6, 2026 13:51
@openshift-ci openshift-ci Bot requested a review from xstefank February 6, 2026 13:57
Copy link
Copy Markdown
Collaborator

@csviri csviri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just comment regarding one minor details, otherwise looks awesome, thank you!


/**
* @deprecated Use {@link LeaderElectionConfigurationBuilder} instead
*/
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add proper annotation not just javadoc

@csviri
Copy link
Copy Markdown
Collaborator

csviri commented Feb 6, 2026

The test failures are not releated, but we have to check on those later

[skip ci]

Signed-off-by: Chris Laprun <[email protected]>
@metacosm metacosm merged commit 280b162 into next Feb 6, 2026
1 check passed
@metacosm metacosm deleted the leader-election branch February 6, 2026 15:22
csviri pushed a commit that referenced this pull request Feb 16, 2026
)

* improve: prepare for removal of exitOnStopLeading from public API

Signed-off-by: Chris Laprun <[email protected]>

* fix: missed deprecated annotation

[skip ci]

Signed-off-by: Chris Laprun <[email protected]>

---------

Signed-off-by: Chris Laprun <[email protected]>
csviri pushed a commit that referenced this pull request Feb 28, 2026
)

* improve: prepare for removal of exitOnStopLeading from public API

Signed-off-by: Chris Laprun <[email protected]>

* fix: missed deprecated annotation

[skip ci]

Signed-off-by: Chris Laprun <[email protected]>

---------

Signed-off-by: Chris Laprun <[email protected]>
csviri pushed a commit that referenced this pull request Mar 6, 2026
)

* improve: prepare for removal of exitOnStopLeading from public API

Signed-off-by: Chris Laprun <[email protected]>

* fix: missed deprecated annotation

[skip ci]

Signed-off-by: Chris Laprun <[email protected]>

---------

Signed-off-by: Chris Laprun <[email protected]>
csviri pushed a commit that referenced this pull request Mar 13, 2026
)

* improve: prepare for removal of exitOnStopLeading from public API

Signed-off-by: Chris Laprun <[email protected]>

* fix: missed deprecated annotation

[skip ci]

Signed-off-by: Chris Laprun <[email protected]>

---------

Signed-off-by: Chris Laprun <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Leader election does not auto-recover after lease acquisition timeout during etcd outage

2 participants